Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Override the soft file descriptor limit #4442

Merged
merged 1 commit into from
Feb 11, 2019

Conversation

pdbain-ibm
Copy link
Contributor

Set the maximum number of file descriptors to the hard limit
for compatibility with the reference implementation.

Fixes #4407
Issue eclipse-omr/omr#3501 covers the necessary OMR changes.

Signed-off-by: Peter Bain peter_bain@ca.ibm.com

Set the maximum number of file descriptors to the hard limit
for compatibility with the reference implementation.

Fixes eclipse-openj9#4407
Issue eclipse-omr/omr#3501 covers the necessary OMR changes.

Signed-off-by: Peter Bain <peter_bain@ca.ibm.com>
@pshipton pshipton added the depends:omr Pull request is dependent on a corresponding change in OMR label Jan 24, 2019
@pdbain-ibm
Copy link
Contributor Author

OMR changes are merged.

@pdbain-ibm
Copy link
Contributor Author

FYI, user-reported issues related to this: #4084 (comment)

Copy link
Member

@DanHeidinga DanHeidinga left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks reasonable

@DanHeidinga
Copy link
Member

Jenkins test sanity plinux jdk8

@DanHeidinga
Copy link
Member

Jenkins test extended xlinux jdk11

@DanHeidinga DanHeidinga self-assigned this Feb 8, 2019
@pshipton
Copy link
Member

pshipton commented Feb 8, 2019

Should we track this as a candidate for a potential 0.12.2 build?

@DanHeidinga
Copy link
Member

Should we track this as a candidate for a potential 0.12.2 build?

Given it's been reported multiple times and is a straightforward fix, I'd say yes.

Add a new label backport:candidate for tracking this

@DanHeidinga DanHeidinga added the backport:candidate Possible candidate for a backport to a `0.x.y+1` release label Feb 11, 2019
@DanHeidinga DanHeidinga merged commit 6058779 into eclipse-openj9:master Feb 11, 2019
@pshipton pshipton removed the backport:candidate Possible candidate for a backport to a `0.x.y+1` release label Apr 16, 2019
@dchopra001 dchopra001 mentioned this pull request Oct 12, 2019
17 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp:vm depends:omr Pull request is dependent on a corresponding change in OMR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Override soft file limit to match reference implementation
3 participants